Skip to content

fix: icon route from '/forms_pro' to '/forms'#162

Merged
harshtandiya merged 1 commit into
bwhtech:developfrom
diniemasjuki:dev
Jul 4, 2026
Merged

fix: icon route from '/forms_pro' to '/forms'#162
harshtandiya merged 1 commit into
bwhtech:developfrom
diniemasjuki:dev

Conversation

@diniemasjuki

@diniemasjuki diniemasjuki commented Jul 3, 2026 •
edited by coderabbitai Bot
Loading

Copy link
Copy Markdown
Contributor

Summary

  • Describe the change and the reason for it.

Testing

  • pre-commit run --all-files
  • cd frontend && yarn typecheck
  • bench --site <site> run-tests --app forms_pro

Checklist

  • PR title follows the conventional format used by this repo
  • Documentation updated if behavior or setup changed
  • Screenshots attached for UI changes when useful

Summary by CodeRabbit

  • Bug Fixes
    • Updated the app’s navigation link so it now opens the forms area at the correct address.
    • This helps users reach the forms section more consistently from the apps screen.

@coderabbitai

coderabbitai Bot commented Jul 3, 2026 •
edited
Loading

Copy link
Copy Markdown

Review Change Stack

📝 Walkthrough

Walkthrough

The route value for the forms_pro app entry in the apps screen configuration is changed from "/forms_pro" to "/forms". No other fields or logic are modified.

Changes

Route Update

Layer / File(s) Summary
App route path update
forms_pro/hooks.py		 The route field of the add_to_apps_screen entry for forms_pro is changed from "/forms_pro" to "/forms".

Estimated code review effort: 1 (Trivial) | ~2 minutes

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Description check ⚠️ Warning The description only repeats the template and does not explain the change, reason, or any testing details. Fill in the Summary with what changed and why, and complete the Testing and Checklist sections with real details.
✅ Passed checks (4 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Title check ✅ Passed The title is concise and accurately describes the main change to the app route.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@harshtandiya harshtandiya changed the title fix icon route from '/forms_pro' to '/forms' fix: icon route from '/forms_pro' to '/forms' Jul 4, 2026
@harshtandiya harshtandiya enabled auto-merge July 4, 2026 18:03
@harshtandiya harshtandiya added this pull request to the merge queue Jul 4, 2026
Merged via the queue into bwhtech:develop with commit 17199d6 Jul 4, 2026
6 of 8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@diniemasjuki @harshtandiya